Skip to content

Add cargo packageURL mining in CI with scancode-action#1

Merged
AyanSinhaMahapatra merged 4 commits into
aboutcode-data:mainfrom
ziadhany:mine-cargo
Sep 29, 2025
Merged

Add cargo packageURL mining in CI with scancode-action#1
AyanSinhaMahapatra merged 4 commits into
aboutcode-data:mainfrom
ziadhany:mine-cargo

Conversation

@ziadhany

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: ziad hany <ziadhany2016@gmail.com>

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ziadhany, LGTM! Please merge with a minor change.

Edit: could you also fix the docs-check errors even though this was there before your PR?

env:
FEDERATEDCODE_GIT_ACCOUNT_URL: https://github.com/aboutcode-data/minecode-data-cargo-test
FEDERATEDCODE_GIT_SERVICE_TOKEN: ${{ secrets.MINING_GITHUB_TOKEN }}
FEDERATEDCODE_GIT_SERVICE_NAME: "the AboutCode bot"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
FEDERATEDCODE_GIT_SERVICE_NAME: "the AboutCode bot"
FEDERATEDCODE_GIT_SERVICE_NAME: "AboutCode Automation"

Could you do the same for pypi?
This was suggested by @keshav-space

scancodeio-repo-branch: "collect-purl-metadata#egg=scancodeio[mining]"
pipelines: "mine_cargo"
env:
FEDERATEDCODE_GIT_ACCOUNT_URL: https://github.com/aboutcode-data/minecode-data-cargo-test

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra merged commit d26d7df into aboutcode-data:main Sep 29, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants